Skip to content
This repository has been archived by the owner on Jan 28, 2022. It is now read-only.

Beefing up SecretScope #101

Merged
merged 44 commits into from
Dec 3, 2019
Merged

Conversation

JasonTheDeveloper
Copy link
Contributor

@JasonTheDeveloper JasonTheDeveloper commented Oct 30, 2019

This PR resolves #99 and in it I've:

  • Check if Secret Scope exists before doing anything.
  • Check for support for ACL in workspace.
  • Check if referenced secret in cluster exists.
  • If acls are not set in config, don't do anything with ACLs.
  • General refactoring and bug fixes.

If secret scope exists or ACL is not supported, report event and remove from reconcile loop.
If the referenced secret does not exist, report event and put back on reconcile loop.

Copy link
Contributor

@Azadehkhojandi Azadehkhojandi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's have a chat with @devlace about ACL at Oneweek :)

controllers/secretscope_controller_databricks.go Outdated Show resolved Hide resolved
controllers/secretscope_controller_databricks.go Outdated Show resolved Hide resolved
controllers/secretscope_controller_databricks.go Outdated Show resolved Hide resolved
controllers/secretscope_controller_databricks.go Outdated Show resolved Hide resolved
stuartleeks added a commit to stuartleeks/azure-databricks-operator that referenced this pull request Nov 25, 2019
Copy link
Contributor

@Azadehkhojandi Azadehkhojandi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

controllers/secretscope_controller_databricks.go Outdated Show resolved Hide resolved
controllers/secretscope_controller.go Outdated Show resolved Hide resolved
controllers/secretscope_controller_databricks.go Outdated Show resolved Hide resolved
@JasonTheDeveloper JasonTheDeveloper merged commit ef5fe9c into Azure:master Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Applying ACL on non Premium Databricks
3 participants